Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 11, 2025

This change improves the handling of the Vitest UI option in the unit test builder. The ui option is now automatically disabled when running in a CI environment to prevent issues with automated pipelines. Additionally, the ui property is only included in the Vitest configuration if it's explicitly a boolean, making the option handling more robust.

Removing the default value for ui from the schema allows the option to be overridden by custom runner configuration files when not explicitly defined in angular.json or via the command line.

This change improves the handling of the Vitest UI option in the unit test builder. The `ui` option is now automatically disabled when running in a CI environment to prevent issues with automated pipelines. Additionally, the `ui` property is only included in the Vitest configuration if it's explicitly a boolean, making the option handling more robust.

Removing the default value for `ui` from the schema allows the option to be overridden by custom runner configuration files when not explicitly defined in `angular.json` or via the command line.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Dec 11, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Dec 12, 2025
@hybrist hybrist added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 12, 2025
@hybrist hybrist merged commit 7c7e6a6 into angular:main Dec 12, 2025
38 checks passed
@hybrist
Copy link
Contributor

hybrist commented Dec 12, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the unit-test/vitest-ui-override branch December 12, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants